js: Always use SOURCE_CONTINUE/SOURCE_REMOVE in later handlers

The constants are more self-explanatory (and therefore readable)
then plain true/false.

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/3950>

diff --git a/js/gdm/authList.js b/js/gdm/authList.js
index 6aa21a5f60860535e0959caa8f8f85a025c127b5..30d8c25f057fc19271a8258acda781c6d0dec3b8 100644 (file)
--- a/js/gdm/authList.js
+++ b/js/gdm/authList.js
@@ -16,6 +16,7 @@
  */
 
 import Clutter from 'gi://Clutter';
+import GLib from 'gi://GLib';
 import GObject from 'gi://GObject';
 import Meta from 'gi://Meta';
 import St from 'gi://St';
@@ -114,7 +115,7 @@ export const AuthList = GObject.registerClass({
             const laters = global.compositor.get_laters();
             laters.add(Meta.LaterType.BEFORE_REDRAW, () => {
                 this._moveFocusToItems();
-                return false;
+                return GLib.SOURCE_REMOVE;
             });
         }
     }

diff --git a/js/gdm/loginDialog.js b/js/gdm/loginDialog.js
index f91ca770ad361d4e7ca111d5b8ce9de73808e76c..4441834f27dab668ebcbc2ba5f006a0116c81ce4 100644 (file)
--- a/js/gdm/loginDialog.js
+++ b/js/gdm/loginDialog.js
@@ -208,7 +208,7 @@ const UserList = GObject.registerClass({
             const laters = global.compositor.get_laters();
             laters.add(Meta.LaterType.BEFORE_REDRAW, () => {
                 this._moveFocusToItems();
-                return false;
+                return GLib.SOURCE_REMOVE;
             });
         }
     }

diff --git a/js/ui/dash.js b/js/ui/dash.js
index 11a9aecc9a39fbc60576ee2d90b52b82340e9e7a..66b3b4a900615de640e98ba3fcba264d77083317 100644 (file)
--- a/js/ui/dash.js
+++ b/js/ui/dash.js
@@ -264,7 +264,7 @@ class ShowAppsIcon extends DashItemContainer {
         const laters = global.compositor.get_laters();
         laters.add(Meta.LaterType.BEFORE_REDRAW, () => {
             AppFavorites.getAppFavorites().removeFavorite(id);
-            return false;
+            return GLib.SOURCE_REMOVE;
         });
 
         return true;
@@ -978,7 +978,7 @@ export const Dash = GObject.registerClass({
                 appFavorites.moveFavoriteToPos(id, favPos);
             else
                 appFavorites.addFavoriteAtPos(id, favPos);
-            return false;
+            return GLib.SOURCE_REMOVE;
         });
 
         return true;

diff --git a/js/ui/main.js b/js/ui/main.js
index 2e85dcebf02e7b8b54adce74800401c0e90136aa..c8cf0c9bb5842b8455018151f81a48227e54af04 100644 (file)
--- a/js/ui/main.js
+++ b/js/ui/main.js
@@ -944,7 +944,7 @@ function _queueBeforeRedraw(workId) {
         const laters = global.compositor.get_laters();
         laters.add(Meta.LaterType.BEFORE_REDRAW, () => {
             _runBeforeRedrawQueue();
-            return false;
+            return GLib.SOURCE_REMOVE;
         });
     }
 }

diff --git a/js/ui/windowManager.js b/js/ui/windowManager.js
index 2dcfbac307095ce00141b4900f8af8970629aa44..66bbe66b0c85c6790828abbd5e4edf9c4ca4d677 100644 (file)
--- a/js/ui/windowManager.js
+++ b/js/ui/windowManager.js
@@ -225,12 +225,12 @@ class WorkspaceTracker {
 
         if (!Meta.prefs_get_dynamic_workspaces()) {
             this._checkWorkspacesId = 0;
-            return false;
+            return GLib.SOURCE_REMOVE;
         }
 
         // Update workspaces only if Dynamic Workspace Management has not been paused by some other function
         if (this._pauseWorkspaceCheck)
-            return true;
+            return GLib.SOURCE_CONTINUE;
 
         for (i = 0; i < this._workspaces.length; i++) {
             const lastRemoved = this._workspaces[i]._lastRemovedWindow;
@@ -290,7 +290,7 @@ class WorkspaceTracker {
         }
 
         this._checkWorkspacesId = 0;
-        return false;
+        return GLib.SOURCE_REMOVE;
     }
 
     keepWorkspaceAlive(workspace, duration) {